Вёрстка ламера № 1. Verder - Макет

[psywalker]

Закончил вёрстку из топика: "Макеты для Вёрстки - Макет Verder" под названием: First - Orange..

Прошу оценить и предъявить за все косяки, буду признателен, если кто нибудь пороется в коде и выявит Даже самые мелкие ошибки

p. s. -Verder , твоё мнение мне особо важно, так как ты имел дело с этим Макетом

Макет: http://pic.ipicture.ru/uploads/090522/WxutJjkOez.png

HTML: http://psywalker.ru/First-Orange/main-page.html

Dimitry Wolotko

Ещё раз увижу пустой топик - буду ругаться.

Дим извини, это я случайно поднажал, как ты так быстро успел, мне буквально Две минуты не хватило, что бы исправить всё)))

Edited May 24, 2009 by psywalker

[AMD]

Вроде как, время 03.30, я выпил, и мне кажется что фон у body не задан(у меня стиль винды принял, цвет точнее).

появился скролер, разрешение 1024х768.

В код еще не смотрел

ушел спать

[Dimitry Wolotko]

Как же вы запарили материться.

AMD - персональное замечание, пока устное.

[psywalker]

AMD

1)

Вроде как, время 03.30, я выпил, и мне кажется что фон у body не задан(у меня стиль винды принял, цвет точнее).

Дело в том, что у body нет фона))

2)

появился скролер, разрешение 1024х768.

Конечно появился, макет заточен под 1024 пиксела

3)

В код еще не смотрел sad.gif

ушел спать нах...

Да куда те смореть, ты бухой в каку, иди спать, завтра позыришь

[SelenIT]

у body нет фона))

А как же это?

body {

...

background: url(../img/body-btm.jpg) 96% 100% no-repeat;

}

Туда же бы и фоновый цвет вписать...

[psywalker]

body {

...

background: url(../img/body-btm.jpg) 96% 100% no-repeat;

}

Туда же бы и фоновый цвет вписать...

Ребят, вы чё, все пьяные сегодня, фон у body БЕЛЫЙ

[SelenIT]

Не белый, а прозрачный (т.к. в сокращенной записи цвет не указан, используется дефолтный, а это transparent) и через него просвечивает дефолтный браузерный фон HTML. У меня-то он белый (как и рассчитано, а вот у AMD, вероятно, нет). Добавление #fff в ту строчку исключило бы такую ситуацию в принципе.

[psywalker]

Не белый, а прозрачный (т.к. в сокращенной записи цвет не указан, используется дефолтный, а это transparent) и через него просвечивает дефолтный браузерный фон HTML. У меня-то он белый (как и рассчитано, а вот у AMD, вероятно, нет). Добавление #fff в ту строчку исключило бы такую ситуацию в принципе.

Фон добавил

[Verder]

Фон добавил

Ну что могу сказать. Очень хорошо! Молоток.

Немного поконтрастней ссылки в разделе "В данный момент..." и будет вообще прекрасно (но тут больше мой косяк, цвет в макете не тот немного дал).

Я код не оценивал, а смотрел глазами заказчика. Он был бы доволен.

Edited May 25, 2009 by Verder

[Nekromancer]

я конечно понимаю, что мыло не настоящее, но зачем там запятая?

Электронная почта: info@firstorange,ru

<script type="text/javascript" src="js/ie7.js"></script>

а это чаво такое?

				 <td class="cur">CHF</td>

				 <td>00.00</td>
				 <td>00.00</td>

место td class="cur" можно тупо поставить th..

[psywalker]

Verder

1)

Ну что могу сказать. Очень хорошо! Молоток.

Если честно, не ожидал такое услышать, думал, что всё на много хуже, спасиб

2)

Немного поконтрастней ссылки в разделе "В данный момент..." и будет вообще прекрасно (но тут больше мой косяк, цвет в макете не тот немного дал).

Да, цвет в макете был именно такой, я даже выбрал самое контрастное место на ссылке в фотошопе, чтобы его, как цвет определить

3)

Я код не оценивал, а смотрел глазами заказчика. Он был бы доволен.

Покажи ему и передай, что я старался

Nekromancer

1)

я конечно понимаю, что мыло не настоящее, но зачем там запятая?

Цитата

Электронная почта: info@firstorange,ru

Так было на макете, но всё равно поменял та Точку

2)

<script type="text/javascript" src="js/ie7.js"></script>

а это чаво такое? blink.gif

Я удивлён, что ТЫ не знаешь, что это такое.)) А это скрипт, который помогает исправить некоторые баги в ИЕ6, и делает работающими некоторые свойства, которые раньше там не пахали..

3)

<td class="cur">CHF</td>

<td>00.00</td>

<td>00.00</td>

место td class="cur" можно тупо поставить th..

Я думал об этом, но тогда бы наверху пришлось бы всё равно делать классы для Двух верхних ячеек th, а во вторых я считаю, что th могут быть только в заголовках

Edited May 25, 2009 by psywalker

[Nekromancer]

Я удивлён, что ТЫ не знаешь, что это такое.)) А это скрипт, который помогает исправить некоторые баги в ИЕ6, и делает работающими некоторые свойства, которые раньше там не пахали..

хз

Я думал об этом, но тогда бы наверху пришлось бы всё равно делать классы для Двух верхних ячеек th, а во вторых я считаю, что th могут быть только в заголовках

Ну считай что название курса это заголовок

[psywalker]

Nekromancer

Я думал об этом, но тогда бы наверху пришлось бы всё равно делать классы для Двух верхних ячеек th, а во вторых я считаю, что th могут быть только в заголовках

Ну считай что название курса это заголовок smile.gif

А это симантично?

[Nekromancer]

в принципе th может быть где угодно, хоть вся таблица из них..

А это симантично?

это уже вам решать

[psywalker]

в принципе th может быть где угодно, хоть вся таблица из них..

Както раз я так сделал, нужно было сделать календарь, где все числа должны были быть жирными и по середине клетки, после этого мне предъявили за это, с тех пор я так больше не делаю))

[Nekromancer]

На самом деле если считать что th должно быть только заголовком, то оно должно всегда располагаться в thead, но никто этого не делает, так что семантика страдает уже изначально

[psywalker]

На самом деле если считать что th должно быть только заголовком, то оно должно всегда располагаться в thead, но никто этого не делает, так что семантика страдает уже изначально wink.gif

Я бы не стал называть thead обязательным элементом, так как в принципи броузеры сами подставляют его, не смотря на его отсутствие, чего не скажешь например про table или tr..и темболее при отсутствии последних могут возникнуть явные проблемы..

[Nekromancer]

thead zне подставляется, а tbody подставляется.(он обязателен, но его можно опускать так как браузеры его сам рисуют).

Я и не просил отказываться от table и tr

[SelenIT]

На самом деле если считать что th должно быть только заголовком, то оно должно всегда располагаться в thead

Насколько я знаю, не обязательно. У th есть опциональный атрибут sсope, показывающий, для чего она является заголовком - для столбца или для строки. По логике из этого следует, что th имеет право на жизнь в любой строке

tbody подставляется.(он обязателен, но его можно опускать так как браузеры его сам рисуют).

Только если кроме него в таблице нет ни thead, ни tfoot, ни других tbody. Если есть хотя бы один - нужно и все остальные указывать явно.

Edited May 25, 2009 by SelenIT

[Nekromancer]

SelenIT, ну ты как всегда даёшь более расширенный ответ.. а мне лень

[psywalker]

Мужики, а ещё что нибудь по коду, да и вообще как сверстал? Можете сказать?

SelenIT

Кстати, поправочка: В таблице можно использовать один из элементов, thead или tfoot, и необязательно, чтобы они были вместе в одной таблице, главное, чтобы с одним из них был обязательный Хотябы один tbody

Edited May 25, 2009 by psywalker

[Nekromancer]

хорошо

[psywalker]

Nekromancer

Слушай,ну а такой вопрос, может некорректный, но всёже: Для фриланса такой уровень ещё не годиться?

[Verder]

Nekromancer

Слушай,ну а такой вопрос, может некорректный, но всёже: Для фриланса такой уровень ещё не годиться?

Отвечу за него если не против (а он сам за себя ответит).

Я скоро сайт студии закончу и буду набирать людей. Напишу.

[SelenIT]

psywalker, не то чтобы придирка, но не лучше ли было сделать иконки в "кредитной линии на любой вкус" фонами? А то перенос текста под них при ужатии ширины окна странно смотрится. Да и кода меньше).

А так код отличный! Конечно, предела совершенству нет, можно еще попытаться избавиться от <div class="line"> (напр. попробовать сделать эту отбивку у предыдущего блока) и др. чисто декоративных пустых элементов (хотя практической пользы в этом немного, больше "научно-спортивный интерес"). Еще есть смысл попробовать уменьшить число фоновых картинок (пособирать что можно в спрайты)... хотя это можно и к придиркам отнести .